Skip to content

chore: update RNTester Cocoapods to 1.13#41248

Closed
okwasniewski wants to merge 1 commit into
facebook:mainfrom
okwasniewski:okwasniewski/chore/gemfile
Closed

chore: update RNTester Cocoapods to 1.13#41248
okwasniewski wants to merge 1 commit into
facebook:mainfrom
okwasniewski:okwasniewski/chore/gemfile

Conversation

@okwasniewski
Copy link
Copy Markdown
Contributor

Summary:

This PR updates the internal version of cocoapods to 1.13, template already uses this version. I've also removed the root folder Gemfile as it's not necessary anymore.

Changelog:

[INTERNAL] [CHANGED] - Update RNTester Cocoapods to 1.13

Test Plan:

Check if cocoapods installs correctly by running:

  1. bundle install
  2. bundle exec pod install

@facebook-github-bot facebook-github-bot added CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Shared with Meta Applied via automation to indicate that an Issue or Pull Request has been shared with the team. labels Oct 30, 2023
@okwasniewski okwasniewski changed the title chore: update RNTester cocoapods to 1.13 chore: update RNTester Cocoapods to 1.13 Oct 30, 2023
@okwasniewski okwasniewski force-pushed the okwasniewski/chore/gemfile branch from 83c7d37 to 6e0388b Compare October 30, 2023 15:53
@okwasniewski
Copy link
Copy Markdown
Contributor Author

When I removed the root Gemfile CI failed so it looks like its still needed

@analysis-bot
Copy link
Copy Markdown

analysis-bot commented Oct 30, 2023

Platform Engine Arch Size (bytes) Diff
android hermes arm64-v8a 17,657,169 -1
android hermes armeabi-v7a n/a --
android hermes x86 n/a --
android hermes x86_64 n/a --
android jsc arm64-v8a 21,034,726 +2
android jsc armeabi-v7a n/a --
android jsc x86 n/a --
android jsc x86_64 n/a --

Base commit: 53a2742
Branch: main

@cipolleschi
Copy link
Copy Markdown
Contributor

Yeah, it's not a simple removal. It should go, but there are other cleanup that are required to remove it properly!

@okwasniewski okwasniewski force-pushed the okwasniewski/chore/gemfile branch from 6e0388b to c028923 Compare November 3, 2023 10:21
@okwasniewski
Copy link
Copy Markdown
Contributor Author

@cipolleschi Can we merge it without removing the root one?

@okwasniewski okwasniewski force-pushed the okwasniewski/chore/gemfile branch from c028923 to 9aadebd Compare November 3, 2023 10:37
@cipolleschi
Copy link
Copy Markdown
Contributor

yes sure. I thought I had already imported this one. Sorry.

@facebook-github-bot
Copy link
Copy Markdown
Contributor

@cipolleschi has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Nov 3, 2023

This pull request was successfully merged by @okwasniewski in b955fc2.

When will my fix make it into a release? | Upcoming Releases

@github-actions github-actions Bot added the Merged This PR has been merged. label Nov 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Merged This PR has been merged. Shared with Meta Applied via automation to indicate that an Issue or Pull Request has been shared with the team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants